Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pac file test #1829

Open
wants to merge 42 commits into
base: master
Choose a base branch
from
Open

Pac file test #1829

wants to merge 42 commits into from

Conversation

bhutianimukul
Copy link

@bhutianimukul bhutianimukul commented Dec 19, 2024

Did the fixes for test case failing and added tests for test coverage
Reference PR
#1787

khushhalm and others added 17 commits November 19, 2024 15:10
* PAC file support via PERCY_PAC_FILE_PATH env var

* Added pac-proxy-agent lib as dependency

* typo fixes

* PR comments addressed

* Moved pac-proxy-agent lib dependency to percy client instead of main package

* Resolved conflicts
* Fix import in ternary statement

* Added comment

* Typo fix in comment

Co-authored-by: ninadbstack <60422475+ninadbstack@users.noreply.github.com>

* Removed trailing space

---------

Co-authored-by: ninadbstack <60422475+ninadbstack@users.noreply.github.com>
* Fix import in ternary statement

* Added comment

* Typo fix in comment

Co-authored-by: ninadbstack <60422475+ninadbstack@users.noreply.github.com>

* Removed trailing space

* Fix import of PERCY_DOM in line w ESM req

* made a minor comment change

---------

Co-authored-by: ninadbstack <60422475+ninadbstack@users.noreply.github.com>
* Fix import in ternary statement

* Added comment

* Typo fix in comment

Co-authored-by: ninadbstack <60422475+ninadbstack@users.noreply.github.com>

* Removed trailing space

* Fix import of PERCY_DOM in line w ESM req

* made a minor comment change

* Added PERCY_FORCE_DIRNAME flag in getPackageJSON util method

* Env var name updated to PERCY_FORCE_EXECUTABLE_DIRNAME

Co-authored-by: ninadbstack <60422475+ninadbstack@users.noreply.github.com>

---------

Co-authored-by: ninadbstack <60422475+ninadbstack@users.noreply.github.com>
* Added PERCY_FORCE_PKG_VALUE in place of PERCY_FORCE_DIRNAME

* Bumped the percy tag to 1.30.3-alpha.4
)

* Added PERCY_FORCE_PKG_VALUE in place of PERCY_FORCE_DIRNAME

* Bumped the percy tag to 1.30.3-alpha.4

* Added PERCY_FORCE_PKG_VALUE in env file and using from there

* Bumped the percy tag to 1.30.3-alpha.5

* fixed linting
@bhutianimukul bhutianimukul requested a review from a team as a code owner December 19, 2024 19:22
@bhutianimukul bhutianimukul requested review from prklm10 and shantanuk-browserstack and removed request for a team December 19, 2024 19:22
@bhutianimukul bhutianimukul changed the base branch from pac-file-support to master December 19, 2024 19:23
@bhutianimukul bhutianimukul changed the base branch from master to pac-file-support December 19, 2024 19:29
@bhutianimukul bhutianimukul changed the base branch from pac-file-support to master December 20, 2024 06:51
@shantanuk-browserstack
Copy link
Contributor

LGTM!

@@ -83,6 +83,8 @@ export class PercyClient {

// Stringifies client and environment info.
userAgent() {
if (this.env.forcedPkgValue) pkg = this.env.forcedPkgValue;
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Add that this is added by LCNC team and why?


if (($# < 1)); then
printf "usage: \n"
printf " ./get_chromuim [-r] rev - will get chromium by revision\n"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this usage correct?


export REVISION=$1

for os in "Linux" "Mac" "Mac_Arm" "Win" "Win_x64";
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

from where this bash file is getting called?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is this something lcnc calling internally?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants